Skip to content

Conversation

@mathiasvr
Copy link
Contributor

Remove old github issue template and add new ones based on those in the main betaflight repo.

Mostly identical to a PR opened it the Betaflight Configurator repo: betaflight/betaflight-configurator#2665.

@haslinghuis haslinghuis added this to the 3.6.0 milestone Nov 25, 2021
@blckmn
Copy link
Member

blckmn commented Nov 26, 2021

AUTOMERGE: (FAIL)

  • github identifies PR as mergeable -> FAIL
  • assigned to a milestone -> PASS
  • cooling off period lapsed -> PASS
  • commit count less or equal to three -> PASS
  • Don't merge label NOT found -> PASS
  • at least one RN: label found -> FAIL
  • Tested label found -> FAIL
  • assigned to an approver -> FAIL
  • approver count at least three -> FAIL

<!-- Specify the version of Betaflight Blackbox Explorer (as displayed in the app) -->
- Blackbox Explorer version:

<!-- If this bug is related to a specific flight controller and/or configuration please specify it here. Create a diff and post it here in a code block. Put (three backticks) at the start and end of the diff block (instructions on how to do a diff: https://oscarliang.com/use-diff-not-dump-betaflight/) -->
Copy link
Member

@ctzsnooze ctzsnooze Nov 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line seems relevant to a flight controller or firmware issue. I wonder if here, for logs it might be more useful to ask the user to post the log header, or a link to the problematic log file itself?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes that makes sense, I updated the PR to specify a log instead.

@mathiasvr mathiasvr force-pushed the update-github-issue-template branch from 517882a to 340a495 Compare December 2, 2021 16:53
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@ctzsnooze ctzsnooze merged commit 72a44db into betaflight:master Dec 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants